Skip to content

fix: docker-compose.prod.yml 경로 및 서버 설정 동기화#55

Merged
Woomin-Wang merged 9 commits into
mainfrom
chore/ci-cd
Apr 22, 2026
Merged

fix: docker-compose.prod.yml 경로 및 서버 설정 동기화#55
Woomin-Wang merged 9 commits into
mainfrom
chore/ci-cd

Conversation

@Woomin-Wang
Copy link
Copy Markdown
Contributor

📝 작업 내용 (Description)

docker-compose.prod.yml 경로 및 서버 설정 동기화

🔄 변경 유형 (Type of Change)

  • ✨ 새로운 기능 (feat)
  • 🐛 버그 수정 (fix)
  • 📝 문서 수정 (docs)
  • 💄 스타일 (style)
  • ♻️ 리팩토링 (refactor)
  • ✅ 테스트 (test)
  • 🔧 기타 (chore)

✅ 체크리스트 (Checklist)

  • 코드가 정상적으로 동작하는지 확인했습니다
  • 기존 테스트가 통과합니다
  • 필요한 경우 새로운 테스트를 추가했습니다

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the production Docker Compose configuration by relocating it to the docker/ directory, adjusting volume paths for Fluent Bit, removing external port access for the PostgreSQL service, and updating the environment file path. A review comment points out that the Fluent Bit volume paths likely need to be updated to reference the parent directory (../fluent-bit/) to correctly locate the configuration files from the new directory structure.

Comment on lines +6 to +7
- ./fluent-bit/fluent-bit.conf:/fluent-bit/etc/fluent-bit.conf:ro
- ./fluent-bit/parsers.conf:/fluent-bit/etc/parsers.conf:ro
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

docker-compose.prod.yml 파일이 docker/ 디렉토리로 이동함에 따라 설정 파일들의 상대 경로를 확인해야 합니다. 만약 fluent-bit 설정 폴더가 프로젝트 루트에 위치한다면, 경로를 ../fluent-bit/...로 수정해야 합니다. 현재 설정된 ./fluent-bit/docker/fluent-bit/ 디렉토리를 가리키므로 파일을 찾지 못할 수 있습니다. 또한 폴더명이 fluentbit에서 fluent-bit으로 변경되었는지도 함께 확인해 주세요.

      - ../fluent-bit/fluent-bit.conf:/fluent-bit/etc/fluent-bit.conf:ro
      - ../fluent-bit/parsers.conf:/fluent-bit/etc/parsers.conf:ro

@Woomin-Wang Woomin-Wang merged commit a89078d into main Apr 22, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant